Skip to content

Public Queries #119

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 131 commits into
base: master
Choose a base branch
from
Open

Public Queries #119

wants to merge 131 commits into from

Conversation

hobbescodes
Copy link
Contributor

@hobbescodes hobbescodes commented May 1, 2025

Description

Task link: https://linear.app/omnidev/issue/OMNI-340/unauth-users-should-be-able-to-see-projects

This PR refactors limitations on rendering for all pages that should be available to be viewed by unauthorized users. The biggest change this PR implements is removing the need for a valid accessToken to perform queries.

Important

Must be tested in combination with omnidotdev/backfeed-api#47

In Addition, other refactors available in this PR can be found here: #140

Test Steps

  1. Verify that logic is sound
  2. Validate that all routes have appropriate access control and render as expected
  3. Validate that state management for CTAs, form elements, etc are appropriate for unauthorized users
  4. Validate that the query option changes are appropriate and work as expected based on the linked PR description

hobbescodes and others added 30 commits April 28, 2025 19:52
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could lock these routes down still if we would like. Happy to adjust!

@hobbescodes hobbescodes marked this pull request as ready for review May 15, 2025 02:49
@@ -11,7 +11,7 @@ import { app } from "lib/config";
import { useOrganizationMembership } from "lib/hooks";

import type { ButtonProps } from "@omnidev/sigil";
import type { Organization } from "generated/graphql";
import { type Organization, useOrganizationQuery } from "generated/graphql";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
import { type Organization, useOrganizationQuery } from "generated/graphql";
import type { Organization } from "generated/graphql";

@@ -11,7 +11,7 @@ import { app } from "lib/config";
import { useOrganizationMembership } from "lib/hooks";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
import { useOrganizationMembership } from "lib/hooks";
import { useOrganizationQuery } from "generated/graphql";
import { useOrganizationMembership } from "lib/hooks";

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants